Skip to content

Ability to exclude timelines from location (https://github.com/ExpDev07/coronavirus-tracker-api/issues/89) + some hotfix with boolean parsing of query params.#90

Merged
ExpDev07 merged 1 commit intomasterfrom
hotfix-timelines
Mar 19, 2020
Merged

Ability to exclude timelines from location (https://github.com/ExpDev07/coronavirus-tracker-api/issues/89) + some hotfix with boolean parsing of query params.#90
ExpDev07 merged 1 commit intomasterfrom
hotfix-timelines

Conversation

@ExpDev07
Copy link
Owner

@ExpDev07 ExpDev07 commented Mar 19, 2020

This introduces the ability to exclude timelines from /v2/locations/:id (#89) by adding ?timelines=0 or ?timelines=false. It will still show timelines by default so not to break existing apps.

Fixes:

  • An issue where all values (even false or 0) were regarded as true when parsed to a boolean in the code was also fixed.

@ExpDev07 ExpDev07 linked an issue Mar 19, 2020 that may be closed by this pull request
@ExpDev07 ExpDev07 merged commit 14b0d20 into master Mar 19, 2020
@ExpDev07 ExpDev07 deleted the hotfix-timelines branch March 19, 2020 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

timelines query argument consistency

1 participant